Initial constant routing support for 7-series #511
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
probot-autolabeler
bot
added
arch-xc7-artix7
litghost
changed the title
litghost
changed the title
litghost
force-pushed
the
constant_routing
branch
from
ddde2c0
to
6b21367
Compare
litghost
added a commit
to litghost/symbiflow-arch-defs
that referenced
this pull request
Mar 28, 2019
This is required to explore global routing errors found in f4pga#511. Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
litghost
added a commit
to litghost/symbiflow-arch-defs
that referenced
this pull request
Mar 28, 2019
This is required to explore global routing errors found in f4pga#511. Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
mkurc-ant
reviewed
Mar 29, 2019
mkurc-ant
reviewed
Mar 29, 2019
Answered in the thread. |
litghost
force-pushed
the
constant_routing
branch
from
6b21367
to
fd70ae8
Compare
probot-autolabeler
bot
added
lang-verilog
litghost
force-pushed
the
constant_routing
branch
from
fd70ae8
to
a40faec
Compare
litghost
changed the title
|
This is now working with the latest VPR from conda. That VPR will be integrated into symbiflow-arch-defs in #518, which is currently failing CI, but will then be merged. There is a latent bug in how the CE graph is expressed that I'm working on. |
mithro
reviewed
Apr 3, 2019
mithro
reviewed
Apr 3, 2019
mithro
reviewed
Apr 3, 2019
mithro
reviewed
Apr 3, 2019
mithro
reviewed
Apr 3, 2019
mithro
approved these changes
Apr 3, 2019
litghost
force-pushed
the
constant_routing
branch
from
a40faec
to
7c2e4af
Compare
I've removed the XXUSEDMUX change for now. I think the approach taken here was wrong, and needs another go. |
litghost
changed the title
litghost
force-pushed
the
constant_routing
branch
2 times, most recently
from
1cb6899
to
5b328e8
Compare
…7-series dedicated constant sources: - Change yosys to use new VCC and GND black boxes for the $true and $false nets. - Implement synthetic tiles BLK_SY-VCC and BLK_SY-GND that connect to a global VCC and GND constant network respectively. - Connect TIEOFF HARD0 and HARD1 pins to the new global constant networks. - Add additional CARRY0 techmap to handle constant CYINIT signals. Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
litghost
force-pushed
the
constant_routing
branch
3 times, most recently
from
d5eca42
to
9a46d2b
Compare
…_tracks Multiple changes are required to add this support: - Temporary bounds on min x_low and y_low are removed. - Added basic test to channel2.Channel and fix bug due to unstable track ordering. - Add track model support to points.decompose_points_into_tracks, in particular adding support for tracks that connect a N-1 if available. - Adds more robust channel verification logic in prjxray_create_edges. - Add "right-only" mode to points.decompose_points_into_tracks. This is useful for arches where all pins are statically determined on the right of the tile, rather than choosen. Signed-off-by: Keith Rothman <537074+litghost@users.noreply.github.com>
litghost
force-pushed
the
constant_routing
branch
from
9a46d2b
to
a21a6f6
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This implements the routing modifications required for constant support in 7-series parts. It is not currently working because the router cannot handle U-shaped routes. Investigations will continue in a seperate branch to isolate the failure and come up with a proposed modifications to this PR and to modify VPR if required.